Skip to content

Update LICENSE.md#39

Open
melvincarvalho wants to merge 1 commit into
SolidOS:mainfrom
melvincarvalho:master
Open

Update LICENSE.md#39
melvincarvalho wants to merge 1 commit into
SolidOS:mainfrom
melvincarvalho:master

Conversation

@melvincarvalho

@melvincarvalho melvincarvalho commented Jun 12, 2026

Copy link
Copy Markdown

Fixes authors in LICENSE.md

@melvincarvalho
Fix copyright: add original author name and correct year
8f8b3f7 
The copyright notice was missing the copyright holder's name and had
the wrong year (2019 instead of 2018). This repo was created on
May 28, 2018 by Melvin Carvalho. The defective notice was added to
a fork in October 2019 and synced back.
TallTed
TallTed reviewed Jun 12, 2026
View reviewed changes
Comment thread LICENSE.md
@@ -1,4 +1,4 @@
Copyright 2019 - present
Copyright 2018 - present Melvin Carvalho and contributors

@TallTed TallTed Jun 12, 2026

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

"present" is not a valid copyright date.

Suggested change
Copyright 2018 - present Melvin Carvalho and contributors
Copyright 2018 - 2026 Melvin Carvalho and contributors

@melvincarvalho

melvincarvalho commented Jul 18, 2026

Copy link
Copy Markdown
Author

"present" is a valid and widely-used copyright convention. It is used verbatim, today, by major open-source projects including Vue (2018-present), Babel (2014-present), Redux (2015-present), and axios (2014-present). It denotes ongoing authorship from the start year to the current one, which is exactly correct for an actively-maintained project.

The substantive fix in this PR is naming the copyright holder. The prior line (Copyright 2019 - present, with no holder named) was defective: a license that identifies no copyright holder is ambiguous and non-compliant, which can jeopardize the project's eligibility for funding and its inclusion in downstream distributions. Correcting it is not cosmetic.

The date format is standard and I'd like to keep it as written. Requesting merge.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

1 more reviewer
@TallTed TallTed TallTed left review comments
Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@melvincarvalho @TallTed